Conversation
| # End agg.iq | ||
|
|
||
| # [CALCITE-6537] Add syntax to allow non-aggregated rows to be used in GROUPING SETS | ||
| # non-aggregated (detail) rows. Requires LENIENT conformance. |
There was a problem hiding this comment.
Since this is not standard SQL and is not supported by mainstream databases, it was converted to equivalent PostgreSQL syntax for verification; the results matched expectations: https://onecompiler.com/postgresql/44yz6y5r4
|
| * | ||
| * @see org.apache.calcite.sql.validate.SqlConformance#isGroupingSetsStarAllowed() | ||
| */ | ||
| class GroupingSetsStarTest { |
There was a problem hiding this comment.
If these tests duplicate the quidem tests, they are not necessary
|
|
||
| # End agg.iq | ||
|
|
||
| # [CALCITE-6537] Add syntax to allow non-aggregated rows to be used in GROUPING SETS |
There was a problem hiding this comment.
the parser uses a stack for grouping star, so it seems to support nested * calls. Are there unit tests for these cases?
We should either have unit tests for the full language supported, or support a more restricted form of the language.
2 participants
jira: https://issues.apache.org/jira/browse/CALCITE-6537